Skip to content

Conversation

@4t8dd
Copy link
Contributor

@4t8dd 4t8dd commented Dec 2, 2025

Adds LogLevel field to VirtualMCPServer CRD OperationalConfig to allow configuring the logging level for vMCP instances. Supports info, debug, warn, and error levels with info as the default.

The deployment builder now passes the --log-level flag to the vmcp container based on the CRD configuration. When not specified, defaults to info level to ensure consistent logging behavior.

Fixes #2781

@github-actions github-actions bot added the size/XS Extra small PR: < 100 lines changed label Dec 2, 2025
@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.63%. Comparing base (efe86fa) to head (1ee71a8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2837      +/-   ##
==========================================
+ Coverage   56.61%   56.63%   +0.01%     
==========================================
  Files         320      320              
  Lines       30932    30934       +2     
==========================================
+ Hits        17513    17520       +7     
+ Misses      11914    11908       -6     
- Partials     1505     1506       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@4t8dd
Copy link
Contributor Author

4t8dd commented Dec 2, 2025

@jhrozek should I bump the crd version as show from the CI ?
EDIT: yeah, found it in the docs. But there is a timeout I can not see the real error of the CI.

@4t8dd 4t8dd force-pushed the issue-2781-vmcp-debug-level branch from d585f76 to 63aaec8 Compare December 2, 2025 12:06
@github-actions github-actions bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Dec 2, 2025
@4t8dd 4t8dd force-pushed the issue-2781-vmcp-debug-level branch from 63aaec8 to 1026a0c Compare December 2, 2025 12:09
@github-actions github-actions bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Dec 2, 2025
@4t8dd
Copy link
Contributor Author

4t8dd commented Dec 2, 2025

yeah, I missed something. I will push it when I fix this.

Adds LogLevel field to VirtualMCPServer CRD OperationalConfig to allow
configuring the logging level for vMCP instances. Supports info, debug,
warn, and error levels with info as the default.

The deployment builder now passes the --log-level flag to the vmcp
container based on the CRD configuration. When not specified, defaults
to info level to ensure consistent logging behavior.

Fixes stacklok#2781

Signed-off-by: 4t8dd <[email protected]>
@4t8dd 4t8dd force-pushed the issue-2781-vmcp-debug-level branch from 1026a0c to f357acf Compare December 2, 2025 14:48
@github-actions github-actions bot added size/S Small PR: 100-299 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Dec 2, 2025
@github-actions github-actions bot added size/S Small PR: 100-299 lines changed and removed size/S Small PR: 100-299 lines changed labels Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Small PR: 100-299 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No way to set debug level through vMCP CRD

1 participant